Use g_clear_pointer to unparent widgets

widget: Don't pass a position to ->size_allocate

The values have been 0/0 for a long time now, so just drop the
GtkAllocation argument and replace it with width and height.

docs: Add GtkEventControllerLegacy

vulkancontext: Add missing error switch

appchooserbutton: Remove priv pointer

testappchooserbutton: Fix for class hierarchy change

appchooserbutton: Inherit from GtkWidget

render: Fix a position/size mixup

Fixes the foreigndrawing demo.

toolbar: Remove dead code

The size definitely changes, otherwise ->size_allocate does not get
called.

toolbar: Remove some unused defines

wayland: Adapt to settings portal api change

ReadAll now accepts an array of patterns.

Merge branch 'scroll-event-propagation' into 'master'

eventcontrollerscroll: Conditionally propagate ::scroll

Closes #45

See merge request GNOME/gtk!250

eventcontrollerscroll: Conditionally propagate ::scroll

Currently, gtk_event_controller_scroll_handle_event() always returns
TRUE if it is handled, which stops the propagation of the event. If
there’s a single GtkEventControllerScroll in the widget hierarchy, that
means that no others will run, depending on the propagation phase. In
Nautilus, this can be observed when adding a scroll controller to the
GtkScrolledWindow (ctrl-scrolling controls the zoom level) - either the
scrolling or the zooming breaks.

Fixes https://gitlab.gnome.org/GNOME/gtk/issues/45

calendar: Remove unused struct member

scrolledwindow: Remove some useless code

This is all being done already be the add_class/remove_class calls
before.

scale: Don't set label css name to "label"

That's already the css name of all labels.

colorscale: Remove x/y arguments from snapshot_scale

We only pass 0 anyway.

Adwaita: use HIG grey for bg_color

- refresh with a light bg color, use the new color palette

Adwaita: fix osd bg color

- don't try deriving from bg_color, osd should be consistent
  for light & dark.

FIxes issue #1449

Adwaita: Propagate bg color to titlebar separator descendants

Gives the same background color to all separators descending from a
title bar than to its direct childrens.

This prevents separators which are in a titlebar but not direct children
from the widget with the titlebar style class from being almost
transparent and hence it prevent them from revealing the clear color of
the window's titlebar (black).

https://gitlab.gnome.org/GNOME/gtk/issues/1231

Merge branch 'wip/jimmac/insensitive-progress' into 'master'

Adwaita: draw insensitive progressbars

See merge request GNOME/gtk!360

Merge branch 'wip/jimmac/hig-colors-gtk4' into 'master'

Adwaita: use new HIG colors

See merge request GNOME/gtk!407

Adwaita: use new HIG colors

- blue selection color, success & destructive colors updates
- blueish tint for dark

Addresses issue #1443

testpopup: Fix GtkEventControllerMotion::motion callback signature

eventcontrollerlegact: Fix ::event docs

As discussed in !388.

icontheme: Stop exporting gtk_icon_info_new_for_file

icontheme: Stop exporting gtk_icon_theme_color_symbolic_pixbuf

icontheme: Drop remaining emblem code

icontheme: Don't reference nonexistent parameter in docs

icontheme: Use a bit field in AsyncSymbolicData

176 -> 144 bytes

icontheme: Stop strdup'ing css colors when loading

... symbolic icons.

gl renderer: Properly retrieve matrix scale

So rotating offscreen nodes works.

widget: Avoid some more ->priv accesses

range: Remove multipress_gesture member

We don't need it in the instance struct since we only set it up and then
access it in its signal callbacks.

widget: Some whitespace fixes

Popover: Minimally document the ::closed signal

This is better than nothing at all. The wording is taken from Carlos's
commit message when he added this shortly before 3.12 (but skip Since).
Skip the bit from his commit message explaining what this replaced; we
don't need to say all the less good things our convenience API replaces.

testtreelistmodel: Don't use non-standard function

as per efd3758f6a779041ff552cb4c5163446fd951368 strcasecmp() is not a C
standard thing (not that we bothered including any header for it anyway)
and so this test failed to build on Windows with Microsoft Visual C.

TreeMenu: Don't manually reinvent g_list_index()

g_list_index() "Gets the position of the element containing the given
data (starting from 0)." That is exactly what we were manually doing.

Merge branch 'wip/3v1n0/recent-info-cleanup' into 'master'

gtk-autocleanups: add cleanup function for GtkRecentInfo

See merge request GNOME/gtk!409

wayland: Fix a thinko in settings portal support

When we decide to fall back because the settings portal
is not present, adhere to that decision elsewhere. And
treat the fontconfig-timestamp like the other special-cased
settings, with G_TYPE_NONE.

gtk-autocleanups: add cleanup function for GtkRecentInfo

Merge branch 'wip/settings-portal' into 'master'

settings portal

See merge request GNOME/gtk!403

wayland: Support the settings portal

Under Wayland, we are currently directly using GSettings
for desktop settings. But in a sandbox, we may not have
access to dconf, so this may fail. Use the new settings
portal instead.

Move the should_use_portal helper to gdk

We want to use it there too, in the future.
Update all callers.

Use a simpler sandbox check

No need to use the runtime dir and allocate a string.
We can just check in /.

Merge branch 'wip/fmuellner/deco-setting' into 'master'

wayland: Remove GNOME Classic-specific workaround

See merge request GNOME/gtk!400

wayland: Remove GNOME Classic-specific workaround

As GSettings now supports session-specific defaults, GNOME Classic
no longer uses a separate schema and the decoration layout is always
determined by the regular schema.

This essentially reverts commit add67b516cb6 (although the code was
moved since then).

https://gitlab.gnome.org/GNOME/gtk/merge_requests/400

GdkQuartz: Handle NULL from [NSGraphicsContext currentContext].

By returning a default surface. The situation where there's no
currentContext arises when GtkCSS is trying to determine the
layout sizes so no actual display is necessary.

Closes: #1411

Use [NSGraphicsContext CGContext] instead of graphicsPort after Yosemite.

Update GDK_OSX versions to current MacOS release.

Update POTFILES.in

Merge branch 'wip/carlosg/public-legacy-controller' into 'master'

Make legacy controller public

See merge request GNOME/gtk!388

Merge branch '1422-gtkentry-s-minimum-width-is-hardcoded-to-150px-GTK4' into 'master'

gtkentry: set minimum-width to 0 instead of 150

Closes #1422 and evince#1002

See merge request GNOME/gtk!393

Updated Slovenian translation

Added Slovenian translation

gtkentry: set minimum-width to 0 instead of 150

and use 150 as natural-width.

Currently there's no way for a GtkEntry to be less
than 150px wide (apart from using "width-chars" property),
this is too much for a default minimum-width, an app
developer may need to have a shorter GtkEntry, for example
when the UI it's been shrunk by the user (see [1]) or when
you want to match the size of another widget (which is less
than 150px) see [2] for Evince bug on using
gtk_combo_box_new_with_model_and_entry() for PDF forms where
GtkEntry of ComboBox is too wide and doesn't match the combo
list width.

Using "width-chars" property may be a workaround to obtain
a short minimum-width for the entry, but is not a proper
solution for the mentioned cases as you may not know how
short your GtkEntry will be, or the fact that using "chars"
as a width unit is not pixel accurate.

Curious note: the commit that introduced the GtkEntry
minimum-width to be 150px is from 20 years ago, see
https://bit.ly/2ySEfK4

[1] This change was already suggested by Benjamin Otte
in a blog comment https://bit.ly/2J96wRo

[2] Fixes issue evince#1002

Merge branch 'nirbheek/meson-fallback-dep-fixes' into 'master'

Misc meson fixes for using subproject dependencies

See merge request GNOME/gtk!232

Update Esperanto translation

Adwaita: prevent devel styling break selection mode

- Selection mode does not get the special devel styling.

- removed teh last-child() selector for it doesn't work anymore.
  Better style all section of the headerbar than none. Proper fix pending.

https://source.puri.sm/Librem5/libhandy/issues/57

quartz: do not cache the screen in the gdkmonitor

Instead we just cache the monitor number and get
out of it the nsscreen when it is needed. This is
a requirement since it nsscreen it is not supposed
to be cached.

Fixes: https://gitlab.gnome.org/GNOME/gtk/issues/1312

Merge branch '1397-gtknotebook-built-in-popup-menu-listing-tabs-doesn-t-use-tab-label-text-for-the-last-tab-4' into 'master'

Notebook: Ensure menu label updates with tab_label

Closes #1397

See merge request GNOME/gtk!386

Merge branch 'lrn/zorder-gtk4' into 'master'

Don't let the OS maintain relative Z-order for windows (GTK4)

See merge request GNOME/gtk!197

tests: Drop "saved positions" test in testgtk

This test solely relies on configure events, which aren't supported anymore.

tests: Drop ::event handler in testgtk

We can use ::size-allocate for it, as window positions in root coordinates
are neither supported nor crucial here.

testsuite: Fix gestures tests for the legacy controller

Let it create one with the right propagation phase, so the event propagation
chain stays the same.

tests: Use GtkEventControllerMotion in testpopup

Track motion through it, instead of a GtkWidget::event handler.

tests: Remove ::event handler in testwindowsize

We just want widget size here, not windowing configuration events.

gtkwidget: Drop obsolete code

This path is not being hit anymore, gtkmain.c does handle those events
itself.

testsuite: Fix gestures test

Was broken by the removal of the default GtkEventControllerLegacy.

gtkeventcontrollerlegacy: Make public

And move ::event into it, dropping the GtkWidget::event signal.

Notebook: Ensure menu_label updates with tab_label

This was noticed in Firefox and demonstrated using a GtkBuilder ui file.
buildable_add_child() calls set_tab_label(), but the latter did nothing
to update the menu_label corresponding to that tab with the new text.
Using Builder to populate the tab child, only tabs other than last got
the right non-default labels, and even that was mostly coincidental, as
adding the main child called update_labels() via real_insert_page(), so
it took effect when the 2nd last main child is added, updating the rest
but leaving the last with the default label, not that given in Builder.

Fix by factoring out the code from child_reordered() to a new helper
menu_item_recreate() and calling that in set_tab_label(), so that
whenever the tab_label is updated, so is its corresponding menu_label.

This fixes the reported case and presumably others that we could write.

fixes https://gitlab.gnome.org/GNOME/gtk/issues/1397

Notebook: Don't notify 2x from set_tab_label_text

It calls set_tab_label(), which already does that.

gtk-demo/main: Suppress implicit fallthru warning

Comments matched to reassure the compiler that fallthrough is
intentional are supposed to precede the case or default keywords, at
least in GCC, so the one here did not suppress the warning with GCC. We
can just the if condition and put the comment at the end to solve that.

https://developers.redhat.com/blog/2017/03/10/wimplicit-fallthrough-in-gcc-7/

.gitignore: Ignore all subdirs in the subprojects dir

meson: Always fetch pangoft2_dep from the pango subproject

This ensures that we will use pangoft2 if it is available, and not if
it is not available.

See: https://gitlab.gnome.org/GNOME/pango/merge_requests/6

meson: Use proxy-libintl subproject when needed and available

Such as on Windows with MSVC.

meson: Add a fallback for the harfbuzz dependency

This meson port is not upstream yet, so a wrap file is not included.
Upstream has expressed interest but the port hasn't been tested on all
platforms yet. Will be added when it gets upstreamed.

Link to WIP port: https://github.com/centricular/harfbuzz

emojichooser: Pass chooser to add_emoji

widget: Stop adding a legacy event controller

The only event signal left is ::event and everything needing that
connects to it directly.

gl renderer: Save some matrix multiplications

We do this for every single node, which is a little costly, especially
since the common case for the modelview matrix these days is a simple
translation. So, check whether the new modelview matrix is only a
translation matrix and if so, don't do a full matrix multiplication per
node.

spinbutton: Don't use gfloat in a code sample

Merge branch 'file-sealing' into 'master'

gdk: seal in-memory files when possible

See merge request GNOME/gtk!353

Merge branch 'gtkbuilder-tag-end-wrong-type' into 'master'

GtkBuildable: Fix the type of the user_data in GtkBuildable.custom_tag_end

See merge request GNOME/gtk!378

gtkimcontextime.c: Fix Korean input

Commit 64a489ad inadvertently introduced a regression that broke Korean
text input because the changes there resulted that only the last input
string that we have from ImmGetCompositionStringW() for each time the
commit signal is emitted is kept, and also as a result the final Korean
character that is input by hitting space is also lost as a result, as we
didn't check for whether we are done with preediting.

Fix these issues by doing the following when we receive the
WM_IME_COMPOSITION message with GCS_RESULTSTR from Windows:
-Do not emit the commit signal during WM_IME_ENDCOMPOSITION, and...
-Emit the commit signal anyways, as we did before c255ba68, however...
-We still save up the string to commit, because we need to re-compute
 the cursor position when we do ->get_preedit_string(), which needs to
 take the GCS_RESULTSTR string we get from WM_IME_COMPOSITION into
 account as well, so that we avoid getting the Pango criticals that
 occur during Chinese (and most likely Japanese) input as the cursor
 position is out-of-range.

Fixes issue #1350.

testsuite/gtk/filterlistmodel.c: Avoid GCCism

Pointer arithmetic on gpointers (void *) is a GCCism, so avoid that by
being more explicit with things.

testsuite/gtk/[flatten|sort]listmodel.c: Avoid VLA usage

Visual Studio is unlikely to support VLAs at any point, so avoid using
them and use g_newa() instead.

GtkBuildable: Fix the type of the user_data in GtkBuildable.custom_tag_end

The previous type was a pointer to a pointer, which seems to be a copy-paste
error from GtkBuildable.custom_tag_start which is an out parameter. It was
always cast in use so this is an API break, but not an ABI one.

Merge branch 'master' into 'master'

gtkstack: fix null pointer dereference

See merge request GNOME/gtk!361

Updated Lithuanian translation

Merge branch 'win32-runtime-immodule-swap-gtk4' into 'master'

GDK W32: Support switching IM contexts at runtime (GTK4)

See merge request GNOME/gtk!372

docs: Update GtkSliceListModel

slicelistmodel: Fix typos in docs

GDK W32: Support switching IM contexts at runtime

This leverages the normal input context switching mechanism in GTK
by making it think that the gtk-im-module setting changed.
The backend returns gtk-im-module value as "ime" if W32
IME API says that an IME is in use. Otherwise it returns
and empty string - this still triggers an input context
switching code, which, not being able to create the desired context
(which is and empty string), falls back to looking at current
keyboard layout (currently that code is still a FIXME).

Paired with the code that signals gtk-im-module change on keyboard layout
switches, this is sufficient to make GTK capable of switching to
the appropriate IM context at runtime. At least, the kinds of context
that specify languages for which they are used automatically by default
(once locale matching is implemented), and the IME context.

Loading other kinds of IM context might still work via specifying
the gtk-im-module setting in gtk ini file, but doing so will likely
make GTK incapable of using the IME context that is used
for Korean, Chinese and Japanese (and some other languages).

Until someone figures out a way to actually change gtk-im-module
setting on Windows at runtime with meaningful values, the behaviour
introduced by this commit seems like a sufficient workaround.

inspector: Highlight CSS node on selection

Fixes #1383

docs: Put subsection into right section

The widget observer APIs belong into the widget docs, not the window
docs. Oops.

video: Add header include guard

Adwaita: fix devel styling for sidebar apps

- don't put the gradient/cogged wheel on apps that have a split headerbar
  due to having a sidebar

Update Esperanto translation

gdk: seal in-memory files when possible

This can be used by compositors to mmap memory without having to
handle SIGBUS.

gtkstack: fix null pointer dereference

The gtk_stack_snapshot_slide() function dereferences the
last_visible_child pointer without proper != NULL ckeck. This might
result in NULL pointer dereference and crash if last_visible_child is
invalid.

Add a != NULL check before dereferencing the pointer.